Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

this introduces an estonian dataset, and removes some of the kosovo data #138

Merged
merged 3 commits into from
Apr 6, 2024

Conversation

pvgenuchten
Copy link
Contributor

@pvgenuchten pvgenuchten commented Apr 5, 2024

creating the PR now, to understand if the cc-by-sa license of this data may be problematic to the mit license? what is your view on that?

relates to #135

image

it also adds a cadastral parcel dataset, an extract from Tartu downtown

image

@pvgenuchten pvgenuchten requested a review from tomkralidis April 5, 2024 17:22
@pvgenuchten
Copy link
Contributor Author

pvgenuchten commented Apr 5, 2024

build failure is solved with #140, suggest to merge that first, then rebase this one

@tomkralidis
Copy link
Member

@pvgenuchten can you fix merge conflicts?

@pvgenuchten pvgenuchten marked this pull request as draft April 5, 2024 19:56
creating the PR now, to understand if the cc-by-sa license of this data may be problematic to the mit license? what is your view on that?
@pvgenuchten pvgenuchten marked this pull request as ready for review April 5, 2024 21:46
@tomkralidis tomkralidis merged commit 854c4b9 into geopython:main Apr 6, 2024
1 check passed
@tomkralidis tomkralidis added this to the FOSS4G Europe 2024 milestone Apr 6, 2024
@ldesousa
Copy link

ldesousa commented Apr 8, 2024

Those two licences are very different and not compatible. You might try to re-share the data with its original licence, separate from the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants